Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add "short block" section to reference page #2641

Merged
merged 2 commits into from
Aug 6, 2024

Conversation

jmegner
Copy link
Collaborator

@jmegner jmegner commented Aug 5, 2024

"short block" is a live feature and is in the cheatsheet but not in the main reference page.

Sidenote: y'all open to the cheatsheet being more explanatory than simply "bounded paragraph"? "short paint" has "Non whitespace sequence stopped by surrounding pair delimeters". I'm thinking "block stopped by surrounding pair delimiters". If yes, I can submit a PR for that.

Checklist

@jmegner jmegner requested a review from a team as a code owner August 5, 2024 19:21
@AndreasArvidsson AndreasArvidsson added this pull request to the merge queue Aug 6, 2024
github-merge-queue bot pushed a commit that referenced this pull request Aug 6, 2024
`"short block"` is a live feature and is in the cheatsheet but not in
the main reference page.

Sidenote: y'all open to the cheatsheet being more explanatory than
simply "bounded paragraph"? `"short paint"` has "Non whitespace sequence
stopped by surrounding pair delimeters". I'm thinking "block stopped by
surrounding pair delimiters". If yes, I can submit a PR for that.


## Checklist

- [ ] I have added
[tests](https://www.cursorless.org/docs/contributing/test-case-recorder/)
- [ ] I have updated the
[docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and
[cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet)
- [ ] I have not broken the cheatsheet

---------

Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
@AndreasArvidsson AndreasArvidsson removed this pull request from the merge queue due to a manual request Aug 6, 2024
@AndreasArvidsson AndreasArvidsson added this pull request to the merge queue Aug 6, 2024
Merged via the queue into main with commit 75b347c Aug 6, 2024
15 checks passed
@AndreasArvidsson AndreasArvidsson deleted the jmegner-short-block-reference-section branch August 6, 2024 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants